Conversation
aec863e to
9b77da7
Compare
...ages/react-native-executorch/common/rnexecutorch/models/object_detection/ObjectDetection.cpp
Outdated
Show resolved
Hide resolved
...ages/react-native-executorch/common/rnexecutorch/models/object_detection/ObjectDetection.cpp
Outdated
Show resolved
Hide resolved
packages/react-native-executorch/src/modules/computer_vision/ObjectDetectionModule.ts
Show resolved
Hide resolved
packages/react-native-executorch/src/modules/computer_vision/ObjectDetectionModule.ts
Show resolved
Hide resolved
|
Documentation CI fails because this PR needs to be committed: #1000. I will also add benchmarks for iOS. Other than that, please review this PR. |
NorbertKlockiewicz
left a comment
There was a problem hiding this comment.
Code looks okay, I ran it on ios and android and it worked as expected, I think we could write some tests for a new parts of object detection API.
packages/react-native-executorch/src/modules/computer_vision/ObjectDetectionModule.ts
Show resolved
Hide resolved
94d1c1b to
61d9548
Compare
…ject_detection/ObjectDetection.cpp
…ject_detection/ObjectDetection.cpp
Co-authored-by: Mateusz Sluszniak <56299341+msluszniak@users.noreply.github.com>
2d33d1d to
7a0e899
Compare
...ages/react-native-executorch/common/rnexecutorch/models/object_detection/ObjectDetection.cpp
Show resolved
Hide resolved
...ages/react-native-executorch/common/rnexecutorch/models/object_detection/ObjectDetection.cpp
Show resolved
Hide resolved
packages/react-native-executorch/src/modules/computer_vision/ObjectDetectionModule.ts
Outdated
Show resolved
Hide resolved
...ages/react-native-executorch/common/rnexecutorch/models/object_detection/ObjectDetection.cpp
Show resolved
Hide resolved
...ages/react-native-executorch/common/rnexecutorch/models/object_detection/ObjectDetection.cpp
Outdated
Show resolved
Hide resolved
packages/react-native-executorch/src/modules/computer_vision/ObjectDetectionModule.ts
Outdated
Show resolved
Hide resolved
…ject_detection/ObjectDetection.cpp
chmjkb
left a comment
There was a problem hiding this comment.
I'll test the demo apps and once I finish i think its ready to 🚢
chmjkb
left a comment
There was a problem hiding this comment.
Two more example app issues - I don't see any bounding boxes in the vision camera demo app. The normal object detection screen works just fine, besides this styling issue which we really need to fix, otherwise I'll go bald trying to click the run model button 😆(you can see it's covered by the navbar), and its not even clickable after i click on the model selector
IMG_9614.MOV
chmjkb
left a comment
There was a problem hiding this comment.
The demo app works now, and the dropdown looks better, buf still using one+ 12 i see the navbar covering the run model button
Description
This PR adds yolo models and multi-method support to our ObjectDetection API.
Deduplication of common CV code will be done in a separate PR
Introduces a breaking change?
Type of change
Tested on
Testing instructions
Screenshots
Related issues
Closes #718
Checklist
Additional notes